Skip to content

bugfix: set/zset/hash member may overflow#2106

Merged
AlexStocks merged 4 commits intoOpenAtomFoundation:unstablefrom
u6th9d:patch-2
Nov 10, 2023
Merged

bugfix: set/zset/hash member may overflow#2106
AlexStocks merged 4 commits intoOpenAtomFoundation:unstablefrom
u6th9d:patch-2

Conversation

@u6th9d
Copy link
Copy Markdown
Contributor

@u6th9d u6th9d commented Nov 7, 2023

No description provided.

@AlexStocks AlexStocks changed the title 修复set/zset/hash成员数量溢出的bug bugfix: set/zset/hash member may overflow Nov 7, 2023
@AlexStocks AlexStocks requested a review from wangshao1 November 9, 2023 09:35
@AlexStocks AlexStocks merged commit 89444f7 into OpenAtomFoundation:unstable Nov 10, 2023
@u6th9d u6th9d deleted the patch-2 branch November 10, 2023 06:27
bigdaronlee163 pushed a commit to bigdaronlee163/pika that referenced this pull request Jun 8, 2024
* Update base_meta_value_format.h

* Update redis_hashes.cc

* Update redis_sets.cc

* Update redis_zsets.cc
cheniujh pushed a commit to cheniujh/pika that referenced this pull request Sep 24, 2024
* Update base_meta_value_format.h

* Update redis_hashes.cc

* Update redis_sets.cc

* Update redis_zsets.cc
@morningli
Copy link
Copy Markdown

我看到这里修复了溢出的问题,但是没有说明为什么会溢出,修复方案也只是出现后报错的方式,现在知道为什么会出现溢出了吗

@Issues-translate-bot
Copy link
Copy Markdown

Bot detected the issue body's language is not English, translate it automatically.


I saw that the overflow problem was fixed here, but it didn’t explain why the overflow occurred. The fix was just to report an error after it occurred. Do you now know why the overflow occurred?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants